Skip to content

chore: fix new clippy lints - #12686

Merged
thibault-martinez merged 3 commits into
developfrom
chore/fix-clippy-lints
Aug 19, 2026
Merged

chore: fix new clippy lints#12686
thibault-martinez merged 3 commits into
developfrom
chore/fix-clippy-lints

Conversation

@thibault-martinez

@thibault-martinez thibault-martinez commented Aug 17, 2026

Copy link
Copy Markdown
Member

Description of change

In prevision of upcoming 1.98.

Fix new clippy lints across several crates:

  • needless_late_init: replace late-initialized let bindings with if/else expressions (overload_monitor, groth16 natives)
  • useless_format: replace format! on literal strings with .to_string() (iota-json-rpc-types, typed-store tests)
  • replace chunks_exact(2) with as_chunks::<2>() (iota-replay)
  • remove redundant super::super::* imports in coin_api tests

🤖 Generated with Claude Code

Fix needless_late_init, useless_format, and manual slice chunking
lints across several crates.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@thibault-martinez
thibault-martinez requested review from a team as code owners August 17, 2026 11:56
@iota-ci iota-ci added dev-tools Issues related to the Developer Tools Team sc-platform Issues related to the Smart Contract Platform group. labels Aug 17, 2026

@sergiupopescu199 sergiupopescu199 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 🚢 (infra owned files)

@thibault-martinez
thibault-martinez added this pull request to the merge queue Aug 19, 2026
Merged via the queue into develop with commit 5fce5e2 Aug 19, 2026
30 checks passed
@thibault-martinez
thibault-martinez deleted the chore/fix-clippy-lints branch August 19, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-tools Issues related to the Developer Tools Team sc-platform Issues related to the Smart Contract Platform group.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants